Skip to content

Conversation

axlerk
Copy link

@axlerk axlerk commented Oct 14, 2012

Перешёл на пробельную табуляцию, т.к. она используется в JSDoc. Можно ли
перемешивать пробелы в комментариях и табы в коде? Валидатор съедает, но неединообразно же.

Перешёл на пробельную табуляцию, т.к. она используется в JSDoc. Можно ли
перемешивать пробелы в комментариях и табы в коде? Валидатор съедает, но
как-то неединообразно...
@azproduction
Copy link
Member

Не нужно мешать. В JSDoc можно тоже табы заюзать (только все может поехать)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А если аргументов будет 50? Подумай как можно сделать нагляднее.

@axlerk
Copy link
Author

axlerk commented Oct 14, 2012

Да, дело именно в том, что всё едет: там всё подогнано пробелами, поэтому их и оставил :)

@axlerk
Copy link
Author

axlerk commented Oct 15, 2012

А можно намекнуть в какую сторону думать?
Мне сейчас в голову приходит вариант получения каждой формы и проход if else, пример для рейтинга:

if (document.form.stars.value === "") {
    document.form.stars.value = 0;
} else {
    document.form.stars.value = +document.form.stars.value;
}

Но я как-то не уверен, что это то, что требуется. :(

@azproduction
Copy link
Member

/**
 * @param {Object} eventData
 * @param {String} eventData.name
 * ...
 */
function Event(eventData) {
    eventData = eventData || {};
    eventData.name = eventData.name || 'Default name';
    // ...
}

Event({
    name: name.
    start: start,
    ...
});

Плюс его еще в том, что аргменты можно передавать в произвольном порядке.

@axlerk
Copy link
Author

axlerk commented Oct 16, 2012

Т.е. функция проверяет, есть ли что-то в eventData, а в Event лежит список объектов-аргументов, там же делается приведение к типу (то, что раньше происходило в return)? Вызывая функцию, мы передаём в неё eventData.name и так далее в любом порядке? Например: function Event(eventData.name, eventData.start) и так далее?

P.S. И ведь пока думал вчера, дошёл до мысли, что в аргументе нужно что-то вроде "Data.*", а уже в звёздочку передавать name, start, которые будут выписаны отдельно. Но не знал как это записывается. :)

@azproduction
Copy link
Member

В функцию будет передаваться уже готовый объект - фактически функция просто валидирует поля

@azproduction azproduction mentioned this pull request Oct 18, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants